Добавил VoiceOver by OneSman7 · Pull Request #1 · akaDuality/AccessibilityTask

OneSman7 · 2021-02-22T14:07:01Z

No description provided.

akaDuality

Приложение с VoiceOver я тоже запустил, все хорошо. Молодец!

akaDuality · 2021-02-27T10:01:12Z

AccessibilityTask/ViewController.swift

+        refreshAccessibilityProperties()
+
+        let argument = email.isValid ? countStepper : emailTextField
+        UIAccessibility.post(notification: .layoutChanged, argument: argument)


Фокус вроде и так останется на поле емейла. Вроде можно попроще:

if email.isValid { UIAccessibility.post(notification: .layoutChanged, argument: argument) }

Паша смотрел сегодня приложение, разобрал весь кейс, можно будет посмотреть в записи, иос там в самом конце.

Вот тут интересно получилось: Паша предлагает после валидации не менять фокус на следующий элемент. У него была открыта клавиатуру Брайля, а после ввода и смены фокуса она сама не закрывается и перестает реагировать на ввод, потому что вводить некуда. Можно просто сообщить, что емейл принят и появились еще элементы.

При этом Паша рассказывает, что скрытие элементов как в задании этой плохой паттерн: даже если элементы недоступны, то про них интересно узнать заранее, чтобы понимать весь сценарий страницы. Но это мой косяк, подумаю как сделать другой тест на нотификейшен .layoutChanged

спасибо за обратную связь!
не удалось сегодня быть утром, обязательно посмотрю видео как только станет доступно.
кажется, тут можно оставить скрытие для обычного ui, а вот VoiceOver "открыть" заранее все элементы.

Такое бывает: адаптируя для VoiceOver мы лучше понимаем как человек строит ментальную модель.

После такого инсайта лучше всего пойти к дизайнеру и обсудить, а нужно ли делать так и для зрячих людей. Вполне возможно, что он согласится с проблемой и ему захочется совсем переделать.

akaDuality · 2021-02-27T10:04:25Z

AccessibilityTask/ViewController.swift

+    fileprivate func getNounForCount(count: UInt, multiple: String, few: String, single: String) -> String {
+
+         let hundredRemnant = count % 100
+         let tenRemnant = count % 10
+
+         if hundredRemnant > 10 && hundredRemnant < 21 {
+             return multiple
+         }
+         else if tenRemnant == 1 {
+             return single
+         }
+         else if tenRemnant > 1 && tenRemnant < 5 {
+             return few
+         }
+
+         return multiple
+     }


Надежней использовать локализацию через .stringsdict

ого) что-то новое)
я уже лет сто использую такой хелпер, а сами слова кладу в strings файлы

Штука в том, что у разных языков разные правила. В английском есть только две формы (rouble, roubles), в русском 3 (1 рубль, 2 рубля, 5 рублей), в других языках может быть и 4-5. .stringsdict дает способ управлять этим всем.

akaDuality · 2021-02-27T10:05:46Z

AccessibilityTask/ViewController.swift

+        sloganLabel.accessibilityLabel = "Поделись кусочком счастья"
+        sloganLabel.accessibilityValue = "Подари кусочек пиццы своему другу. Напиши адрес и оплати, а мы пришлем ему письмо где получить."


Если брать текст из лейблов, то не будет дублирования и текст всегда будет одинаковый для зрячих и незрячих

я тоже так подумал сначала. а потом мне VoiceOver прочел как было в лейбле - "счастьяяяяяя".
тут уже подумалось, что тексты для VoiceOver могут отличаться от значений лейблов.
но, видимо, это лишь исключение из правил было

Возможно, «счастьяяяя» это перебор и для обычных людей :-)

akaDuality · 2021-02-27T10:07:01Z

AccessibilityTask/ViewController.swift

+
+        emailTextField.isAccessibilityElement = true
+        emailTextField.accessibilityLabel = "Адрес друга"
+        emailTextField.accessibilityTraits.insert(.header)


Скорее всего поле ввода не должно быть хедером. Понимаю, что рядом с ним надпись была хедером, но Паша говорил, что это непривычно.

понятно. не хотело надпись отдельно как заголовок давать ибо она без поля не имеет смысла

Отделять и правда не нужно, Паша сказал, что в этом смысле поле сделано очень круто. Просто .header для него лишний.

Офтоп вопрос: ты впервые адаптируешь приложение или уже был опыт?

для VoiceOver первый раз. для Dynamic Type уже около года адаптируем, правда только новые экраны. поэтому не все еще его поддерживают. Правда и там ничего не делали пока с кнопками и иконками (иконки только убираем иногда ибо на больших размерах текста от них толку ноль часто).

akaDuality · 2021-02-27T10:07:53Z

AccessibilityTask/ViewController.swift

+        paymentButton.accessibilityLabel = "Оплатить"
+
+        successTitleLabel.isAccessibilityElement = true
+        successTitleLabel.accessibilityLabel = successTitleLabel.text! + " " + successCountLabel.text!


Лучше через запятую, это даст легкую паузу

akaDuality · 2021-02-27T10:10:19Z

AccessibilityTask/ViewController.swift

+
+    func refreshAccessibilityFrames() {
+
+        sloganLabel.accessibilityFrame = sloganLabel.frame.union(descriptionLabel.frame)


Если не конвертируешь фрейм в координаты экрана вручную, то лучше использовать accessibilityFrameInContainerSpace. Сейчас оно тоже работает, потому что в скролвью не завернуто и координаты совпадают.

Вижу, что в других местах конвертируешь.

я его искал, но он только у UIAccessibilityElement есть, а вот у вьют нету(
да, здесь тоже стоило на всякий в экранные координаты переводить

akaDuality · 2021-02-27T10:11:07Z

AccessibilityTask/ViewController.swift

+        }
+
+        if let postcardFrame = screenFrame(for: postcardContainer),
+           let switchCenter = screenCenter(for: addPostcardSwitch) {


Оу, а я даже не обращал внимание, что это в координатах экрана надо. Спасибо!

akaDuality · 2021-02-27T10:13:21Z

AccessibilityTask/ViewController.swift

+        if paymentButton.isEnabled {
+            paymentButton.accessibilityHint = "Можно выполнить из любого места через Magic Tap. Коснитесь двумя пальцами дважды."
+        }
+        else if emailMistakeLabel.isHidden {
+            paymentButton.accessibilityHint = "Введите адрес друга"
+        }
+        else {
+            paymentButton.accessibilityHint = emailMistakeLabel.text
+        }


Очень хорошо

akaDuality · 2021-02-27T10:14:26Z

AccessibilityTask/ViewController.swift

+        successTitleLabel.accessibilityFrame = UIAccessibility.convertToScreenCoordinates(successFrame, in: successModalView)
+    }
+
+    func refreshAccessibilityProperties() {


Я бы только постарался разбивать на маленькие функции и применять их рядом с основным кодом, так будет чуть проще поддерживать.

у меня что-то такое, но в зачатке. разделил на то, что устанавливается один раз, то, что меняется, и фреймы.
дальше, как говорится, можно больше) но тут слишком простое приложение

akaDuality · 2021-02-28T11:18:45Z

AccessibilityTask/ViewController.swift

+        refreshAccessibilityProperties()
+
+        let argument = email.isValid ? countStepper : emailTextField
+        UIAccessibility.post(notification: .layoutChanged, argument: argument)


Паша смотрел сегодня приложение, разобрал весь кейс, можно будет посмотреть в записи, иос там в самом конце.

Вот тут интересно получилось: Паша предлагает после валидации не менять фокус на следующий элемент. У него была открыта клавиатуру Брайля, а после ввода и смены фокуса она сама не закрывается и перестает реагировать на ввод, потому что вводить некуда. Можно просто сообщить, что емейл принят и появились еще элементы.

При этом Паша рассказывает, что скрытие элементов как в задании этой плохой паттерн: даже если элементы недоступны, то про них интересно узнать заранее, чтобы понимать весь сценарий страницы. Но это мой косяк, подумаю как сделать другой тест на нотификейшен .layoutChanged

akaDuality · 2021-02-28T11:20:47Z

AccessibilityTask/ViewController.swift

+        }
+        else {
+            let emailText = emailTextField.text ?? ""
+            emailTextField.accessibilityValue = emailMistakeLabel.text! + ", " + emailText


Тоже с разбора домашки: в value текстового поля должен быть только текст. Если туда добавить что-то свое, то текстовое поле ломается и предлагает поменять и текст нашей ошибки в том числе. Предлагаю ошибку дописывать в .accessibilityLabel

но accessibilityLabel вроде не должен меняться или я неправильно понял?

Если фокус не меняется, то лейбл и правда не читается, а меняется value после действия, при изменении значения: у степперов и слайдеров, полей ввода.

Тут получается, что value уже занят текстом внутри, поэтому придется писать как-то иначе. Остается только лейбл. При этом, если мы снова поставим фокус на поле ввода, то лейбл прочитается, а если мы завершили редактирование, то лучше сообщить о неправильной валидации через оповещение.

спасибо, понял
все таки без "прослушивания" всех веток сценария самому качественный experience не сделать :)

Добавил VoiceOver

7efea84

akaDuality approved these changes Feb 27, 2021

View reviewed changes

akaDuality approved these changes Feb 28, 2021

View reviewed changes

		sloganLabel.accessibilityLabel = "Поделись кусочком счастья"
		sloganLabel.accessibilityValue = "Подари кусочек пиццы своему другу. Напиши адрес и оплати, а мы пришлем ему письмо где получить."


		func refreshAccessibilityFrames() {

		sloganLabel.accessibilityFrame = sloganLabel.frame.union(descriptionLabel.frame)

Conversation

OneSman7 commented Feb 22, 2021

Uh oh!

akaDuality left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants